Skip to content

fix: allow multiple users to access shared dataset - #634

Open
gbhavya487-ux wants to merge 1 commit into
jsbroks:masterfrom
gbhavya487-ux:fix/multiple-user-access
Open

gbhavya487-ux wants to merge 1 commit into
jsbroks:masterfrom
gbhavya487-ux:fix/multiple-user-access

Conversation

@gbhavya487-ux

Copy link
Copy Markdown

Summary

  • Fix admin user creation response to return the newly created user
  • Add a test verifying multiple users can access a shared dataset

Testing

  • python -m pytest backend/tests/api/test_dataset.py -v
  • 1 passed

@gbhavya487-ux

Copy link
Copy Markdown
Author

This issue has been fixed and the changes have been merged in PR #634. Thanks!

@gbhavya487-ux

Copy link
Copy Markdown
Author

Hi, just following up on this PR. The checks are passing and it’s ready for review. Whenever you have time, could you please take a look? Thank you!

@gbhavya487-ux

Copy link
Copy Markdown
Author

Hi, just following up on this PR. It has been open for about 10 days. When you have a chance, could you please review it? Thank you!

@gbhavya487-ux

Copy link
Copy Markdown
Author

Hi maintainers, I wanted to follow up once more on this PR. It has been open for about a month without a review. Please let me know if any changes or additional tests are needed from my side. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant